\label{todo__todo000001}
\hypertarget{todo__todo000001}{}
 
\begin{DoxyDescription}
\item[Member \hyperlink{class_a_v_i_file_a590b7a973a107c19b0d5904872e645ee}{AVIFile::AVIFile}() ]mainHdr not initialized 

add checking for NULL pointers



add checking for NULL pointers


\end{DoxyDescription}

\label{todo__todo000005}
\hypertarget{todo__todo000005}{}
 
\begin{DoxyDescription}
\item[Member \hyperlink{class_a_v_i_file_a468342fdddffd3aa4e3d33ed2232193d}{AVIFile::GetFrame}(\hyperlink{class_frame}{Frame} $\ast$frame, int frameNum) ]use 64 bit seek 

we actually don't need the frame here, we could use just a void pointer 
\end{DoxyDescription}

\label{todo__todo000004}
\hypertarget{todo__todo000004}{}
 
\begin{DoxyDescription}
\item[Member \hyperlink{class_a_v_i_file_ad18c3fdbcae2b439987c31803cdabb75}{AVIFile::GetFrameInfo}(off\_\-t \&offset, int \&size, int frameNum) ]the size parameter is redundant. All frames have the same size, which is also in the mainHdr. 

all index related operations should be isolated 
\end{DoxyDescription}

\label{todo__todo000003}
\hypertarget{todo__todo000003}{}
 
\begin{DoxyDescription}
\item[Member \hyperlink{class_a_v_i_file_ae3cf3009dfda33f25eaef7eed374c18c}{AVIFile::Init}(int format, int sampleFrequency, int indexType) ]consolidate \hyperlink{class_a_v_i_file_ae3cf3009dfda33f25eaef7eed374c18c}{AVIFile::Init}, \hyperlink{class_a_v_i1_file_a10a9e391cfe38e75273b05af17ec8a54}{AVI1File::Init}, \hyperlink{class_a_v_i2_file_a2bcacacbf565fddffa27bd8be90aa62c}{AVI2File::Init}. They are somewhat redundant. 
\end{DoxyDescription}

\label{todo__todo000006}
\hypertarget{todo__todo000006}{}
 
\begin{DoxyDescription}
\item[Member \hyperlink{class_a_v_i_file_ac35dd7e4beb3da69fa6bbb79590f72e5}{AVIFile::PrintDirectoryEntryData}(const \hyperlink{class_r_i_f_f_dir_entry}{RIFFDirEntry} \&entry) ]use 64 bit routines 
\end{DoxyDescription}

\label{todo__todo000007}
\hypertarget{todo__todo000007}{}
 
\begin{DoxyDescription}
\item[Member \hyperlink{class_d_v_frame_a26b2006ed9deae132cf9a2f7ef683d7c}{DVFrame::IsPAL}(void) ]function can't handle \char`\"{}empty\char`\"{} frame 
\end{DoxyDescription}

\label{todo__todo000008}
\hypertarget{todo__todo000008}{}
 
\begin{DoxyDescription}
\item[Member \hyperlink{class_r_i_f_f_file_a29c1a27664c322f7d6b3d9f5c12e4036}{RIFFFile::FindDirectoryEntry}(FOURCC type, int n=0) const  ]inefficient if the directory has lots of items 
\end{DoxyDescription}